GizmoSQL: switch from gizmosqlline (Java) to gizmosql_client#863
Merged
rschu1ze merged 3 commits intoClickHouse:mainfrom May 7, 2026
Merged
GizmoSQL: switch from gizmosqlline (Java) to gizmosql_client#863rschu1ze merged 3 commits intoClickHouse:mainfrom
rschu1ze merged 3 commits intoClickHouse:mainfrom
Conversation
Use the new gizmosql_client CLI shell (shipped with the gizmosql server release) instead of the Java-based gizmosqlline client. This removes the openjdk-17 install and the separate gizmosqlline download. - benchmark.sh / run.sh: replace gizmosqlline (-u JDBC-URI -n -p -f) with "gizmosql_client --file ...", which reads connection settings from the GIZMOSQL_HOST / GIZMOSQL_PORT / GIZMOSQL_USER / GIZMOSQL_PASSWORD env vars. - run.sh: prepend ".timer on" and ".mode trash" so the client emits one "Run Time: X.XXs" line per statement (to stderr) without rendering result rows; merge stderr into stdout for log capture. - benchmark.sh: update the result parser regex from sqlline-style "rows? selected ([0-9.]+ seconds)" to "Run Time: [0-9.]+s". - util.sh: rename GIZMOSQL_USERNAME -> GIZMOSQL_USER and switch from a JDBC URI to host/port env vars (the names gizmosql_client reads natively). See https://docs.gizmosql.com/#/client for client usage details.
prmoore77
force-pushed
the
gizmosql-cli-client
branch
from
b44aadd to
1dd4994
Compare
prmoore77
changed the title
prmoore77
force-pushed
the
gizmosql-cli-client
branch
from
1dd4994 to
b44aadd
Compare
prmoore77
changed the title
rschu1ze
reviewed
May 7, 2026
| # Install requirements | ||
| apt-get update -y | ||
| apt install openjdk-17-jre-headless unzip netcat-openbsd -y | ||
| apt install unzip netcat-openbsd -y |
Member
There was a problem hiding this comment.
L. 7 and 8 need sudo, please check the other invocations of apt in the repository.
Contributor
Author
There was a problem hiding this comment.
hi @rschu1ze - this should be fixed now.
| fi | ||
|
|
||
| # Server setup Install | ||
| # Install the GizmoSQL server and client (gizmosql_client is the CLI shell) |
Member
There was a problem hiding this comment.
L. 21: Please don't copy binaries into /usr/local/bin/. Instead, please copy to a local directory and modify $PATH so that it finds the binary.
Contributor
Author
There was a problem hiding this comment.
hi @rschu1ze - this should be fixed now.
- Use sudo for apt invocations - Install gizmosql binaries into a local ./bin and prepend to PATH instead of copying to /usr/local/bin Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
rschu1ze
reviewed
May 7, 2026
| # Start the GizmoSQL server | ||
| start_gizmosql | ||
|
|
||
| # Enable timer and discard result rows (we only care about Run Time) |
Member
There was a problem hiding this comment.
Please change l. 22 to
echo 3 | sudo tee /proc/sys/vm/drop_caches > /dev/null
I got tee: /proc/sys/vm/drop_caches: Permission denied errors locally.
rschu1ze
approved these changes
May 7, 2026
Without sudo, tee fails with "Permission denied" writing to /proc/sys/vm/drop_caches. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
Author
|
🚀🚀🚀🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Switches the GizmoSQL ClickBench scripts from the Java-based
gizmosqllineSQLLine client to the new C++gizmosql_clientshell that ships in the same release archive asgizmosql_server.openjdk-17-jre-headlessapt install and the separategizmosqllinebinary download —gizmosql_clientis already ingizmosql_cli_linux_${ARCH}.zip.gizmosqlline -u <jdbc-uri> -n -p -finvocations withgizmosql_client --file ..., which reads connection parameters from theGIZMOSQL_HOST/GIZMOSQL_PORT/GIZMOSQL_USER/GIZMOSQL_PASSWORDenv vars natively.run.sh, prepends.timer onand.mode trashto each per-query batch so the client emits oneRun Time: X.XXsline per statement (to stderr) and skips rendering result rows. stderr is merged into stdout for log capture.benchmark.shfrom sqlline'srows? selected ([0-9.]+ seconds)toRun Time: [0-9.]+s.See https://docs.gizmosql.com/#/client for full client usage.
Test plan
gizmosql_server/gizmosql_clientv1.22.4 on a small in-memory table; theRun Time: X.XXsparser produced the expected[t1, t2, t3],output rows.